chore: increase resource class to xlarge for CircleCI jobs by rodrigopavezi · Pull Request #1698 · RequestNetwork/requestNetwork

rodrigopavezi · 2026-02-11T14:36:28Z

Description of the changes

Upgraded CircleCI resource classes from large to xlarge for the test-unit and integration test jobs, and added an xlarge resource class to the third test job to improve test execution performance.

Summary by CodeRabbit

Chores
- Increased CI job resource allocation to speed up and stabilize test runs.
Tests
- Made integration tests resilient to external service outages by detecting availability and skipping affected tests and cleanup when unreachable.
- Increased test timeouts to allow longer-running asynchronous operations and reduce flaky failures.

coderabbitai · 2026-02-11T14:36:41Z

Walkthrough

CircleCI job resource allocations increased to xlarge; Lit Protocol integration tests now detect Lit network availability and skip/adjust cleanup when unreachable; several request-client.js tests had Jest timeouts increased to 180000 ms.

Changes

Cohort / File(s)	Summary
CircleCI Configuration `.circleci/config.yml`	Updated `resource_class` values: changed `test-unit` and `test-integration-with-request-node` from `large` to `xlarge`, and added `resource_class: xlarge` for `test-nightly`. No workflow/steps logic changed.
Integration Tests (Lit Protocol) `packages/integration-test/test/lit-protocol.test.ts`	Added `litNetworkAvailable` flag and `skipIfNoLitNetwork` helper; wrapped `litProvider.initializeClient()` in try/catch, set flag on failure, log warning, early-return/skip tests when network unreachable, and conditionally attempt cleanup in `afterAll`. No public API changes.
Request-client.js Tests (Jest timeouts) `packages/request-client.js/test/index.test.ts`	Increased global Jest timeout and multiple individual test timeouts from shorter values (15–60s) to `180000` ms to allow longer async operations. No test logic changes.

Sequence Diagram(s)

mermaid
sequenceDiagram
autonumber
participant TestRunner as Test Runner
participant LitProvider as Lit Provider Client
participant LitNetwork as Lit Network
rect rgba(100,149,237,0.5)
TestRunner->>LitProvider: initializeClient()
LitProvider->>LitNetwork: connect / handshake
alt connection succeeds
LitNetwork-->>LitProvider: success
LitProvider-->>TestRunner: initialized
TestRunner->>TestRunner: run gated tests (encrypt/decrypt, requests)
TestRunner->>LitProvider: perform cleanup
LitProvider->>LitNetwork: cleanup calls
else connection fails
LitNetwork-->>LitProvider: error/timeout
LitProvider-->>TestRunner: throw / return failure
TestRunner->>TestRunner: set litNetworkAvailable = false
TestRunner->>TestRunner: skip gated tests
TestRunner->>LitProvider: attempt conditional cleanup (no-op if uninitialized)
end
end

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: upgrading CircleCI resource classes to xlarge, which is the primary focus of the .circleci/config.yml changes.
Description check	✅ Passed	The description provides clear explanation of the changes, covering upgraded resource classes for test jobs and the objective of improving test execution performance, matching the template requirement.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch fix/ci

No actionable comments were generated in the recent review. 🎉

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

rodrigopavezi · 2026-02-11T14:36:43Z

This stack of pull requests is managed by Graphite. Learn more about stacking.

greptile-apps · 2026-02-11T15:12:16Z

Greptile Overview

Greptile Summary

This PR improves CI/CD performance and test reliability by upgrading CircleCI job resources and addressing test flakiness. The changes include:

CircleCI Configuration: Upgraded resource class from large to xlarge for test-unit and two integration test jobs, providing more CPU and memory for faster test execution
Lit Protocol Integration Tests: Implemented network availability detection with graceful test skipping when the external Lit Protocol network (datil-dev) is unreachable, preventing test failures due to external service outages
Request Client Tests: Extended test timeouts from 15-60 seconds to 180 seconds to accommodate longer-running asynchronous operations

All changes are defensive improvements that make the CI pipeline more stable and performant without affecting production code.

Confidence Score: 5/5

This PR is safe to merge with no risks identified
All changes are limited to CI configuration and test infrastructure. The resource class upgrades are straightforward and won't negatively impact functionality. The test resilience improvements (network availability checks and timeout extensions) follow best practices for handling external dependencies and async operations. No production code is modified.
No files require special attention

Important Files Changed

Filename	Overview
.circleci/config.yml	Upgraded resource class from `large` to `xlarge` for three test jobs to improve performance
packages/integration-test/test/lit-protocol.test.ts	Added network availability detection with graceful test skipping when Lit Protocol network is unreachable
packages/request-client.js/test/index.test.ts	Extended test timeouts from 15-60s to 180s for async operations to reduce flakiness

greptile-apps

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

MantisClone · 2026-02-11T22:13:41Z

@greptile review

greptile-apps

_{3 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

bassgeta

Looking good, hope this solves all of our flaky tests 🤞

rodrigopavezi · 2026-02-12T13:06:10Z

Merge activity

Feb 12, 1:06 PM UTC: A user started a stack merge that includes this pull request via Graphite.
Feb 12, 1:06 PM UTC: Graphite rebased this pull request as part of a merge.
Feb 12, 1:07 PM UTC: @rodrigopavezi merged this pull request with Graphite.

…y checks

…ailability

…lability

…tability

…s for consistent re-runs

rodrigopavezi marked this pull request as ready for review February 11, 2026 14:41

github-actions bot requested review from MantisClone, alexandre-abrioux-rf, bassgeta and dave-rf February 11, 2026 14:41

rodrigopavezi requested review from aimensahnoun and removed request for alexandre-abrioux-rf and dave-rf February 11, 2026 14:42

greptile-apps bot reviewed Feb 11, 2026

View reviewed changes

bassgeta approved these changes Feb 12, 2026

View reviewed changes

aimensahnoun approved these changes Feb 12, 2026

View reviewed changes

rodrigopavezi added 6 commits February 12, 2026 13:06

chore: increase resource class to xlarge for CircleCI jobs

c05da4c

test: enhance Lit Protocol integration tests with network availabilit…

95ec0cc

…y checks

test: implement skip logic for Lit Protocol tests based on network av…

01bf463

…ailability

test: ensure cleanup in Lit Protocol tests regardless of network avai…

a9ac6e1

…lability

test: extend timeout durations in request-client tests for improved s…

5185f3f

…tability

test: reset litNetworkAvailable flag in Lit Protocol integration test…

ea4d569

…s for consistent re-runs

rodrigopavezi force-pushed the fix/ci branch from 4e54b3d to ea4d569 Compare February 12, 2026 13:06

rodrigopavezi merged commit a5d6a48 into master Feb 12, 2026
2 of 3 checks passed

rodrigopavezi deleted the fix/ci branch February 12, 2026 13:07

This was referenced Feb 12, 2026

SDK - Migrate Lit Protocol from Datil to Naga network before 2/25 shutdown #1702

Open

SDK - CircleCI test jobs failing intermittently due to resource constraints and external dependencies #1703

Closed

rodrigopavezi mentioned this pull request Feb 16, 2026

chore: publish 0.62.0 #1705

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: increase resource class to xlarge for CircleCI jobs#1698

chore: increase resource class to xlarge for CircleCI jobs#1698
rodrigopavezi merged 6 commits intomasterfrom
fix/ci

rodrigopavezi commented Feb 11, 2026 •

edited by MantisClone

Loading

Uh oh!

coderabbitai bot commented Feb 11, 2026 •

edited

Loading

Uh oh!

rodrigopavezi commented Feb 11, 2026 •

edited

Loading

Uh oh!

greptile-apps bot commented Feb 11, 2026 •

edited

Loading

Uh oh!

greptile-apps bot left a comment

Uh oh!

MantisClone commented Feb 11, 2026

Uh oh!

greptile-apps bot left a comment

Uh oh!

bassgeta left a comment

Uh oh!

rodrigopavezi commented Feb 12, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

rodrigopavezi commented Feb 11, 2026 • edited by MantisClone Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of the changes

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Uh oh!

rodrigopavezi commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

greptile-apps bot commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Overview

Greptile Summary

Confidence Score: 5/5

Important Files Changed

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

MantisClone commented Feb 11, 2026

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

bassgeta left a comment

Choose a reason for hiding this comment

Uh oh!

rodrigopavezi commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

rodrigopavezi commented Feb 11, 2026 •

edited by MantisClone

Loading

coderabbitai bot commented Feb 11, 2026 •

edited

Loading

rodrigopavezi commented Feb 11, 2026 •

edited

Loading

greptile-apps bot commented Feb 11, 2026 •

edited

Loading

rodrigopavezi commented Feb 12, 2026 •

edited

Loading